Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

도메인 entity, repository 세팅 #37

Merged
merged 2 commits into from
Aug 16, 2022
Merged

도메인 entity, repository 세팅 #37

merged 2 commits into from
Aug 16, 2022

Conversation

Kimakjun
Copy link
Collaborator

@Kimakjun Kimakjun commented Aug 14, 2022 •
edited
Loading

도메인 entity, repository 세팅

  • 도메인 entity 는 일단 기획이 구체화되면서 변경사항이 필요할 수도 있는데 이부분은 각자 작업자가 수정하면서 진행하면 될 것 같습니다.

스웨거 설정 변경.

  • 컨트롤러 전역 api prefix 추가
  • 이제 도메인 별로 폴더 구조를 변경하였기 때문에 특정 package 로 스웨거 api 노출을 설정할 수 없기때문에 paths 설정에서 /api/** 으로 스웨거 api 노출을 지정하였습니다.
  • 스웨거에서 디폴트로 노출시키는 의미없는 error 컨트롤러 api 노출을 제외하기 위해서 paths 설정에서 "/api/error.*" 부분을 추가했습니다.

ISSUE

@Kimakjun
- 스웨거 paths 설정 변경
84f6a84 
- 전역 api prefix 적용
- 불필요한 코드 삭제
@Kimakjun
- entity repository setting
f2f8ccb
@Kimakjun Kimakjun requested a review from snanoh August 14, 2022 06:18
@Kimakjun Kimakjun self-assigned this Aug 14, 2022
@snanoh
Copy link
Collaborator

snanoh commented Aug 16, 2022

고생하셨습니다~ 😁

@Kimakjun Kimakjun merged commit 9e0d665 into develop Aug 16, 2022
@Kimakjun Kimakjun deleted the feature/entity-setting branch August 16, 2022 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@snanoh snanoh Awaiting requested review from snanoh

Assignees

@Kimakjun Kimakjun

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Kimakjun @snanoh